Skip to content

Fix segfault with undefined relevant_snapshot in Iceberg metadata#1211

Merged
zvonand merged 1 commit intoantalya-25.8from
bugfix/antalya-25.8/fix_segfaul_system_table_glue_catalog
Dec 11, 2025
Merged

Fix segfault with undefined relevant_snapshot in Iceberg metadata#1211
zvonand merged 1 commit intoantalya-25.8from
bugfix/antalya-25.8/fix_segfaul_system_table_glue_catalog

Conversation

@ianton-ru
Copy link

Changelog category (leave one):

  • Critical Bug Fix (crash, data loss, RBAC) or LOGICAL_ERROR

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Solved #1210
Fix segfault in requests to system,table when table in Iceberg catalog does not have snapshot.

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Performance tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All with Aarch64
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • Tiered Storage (2h)

@github-actions
Copy link

Workflow [PR], commit [4619924]

@ianton-ru
Copy link
Author

Failed test - 03644_object_storage_correlated_subqueries unrelated (fix in #1203)

@zvonand zvonand merged commit 8370e98 into antalya-25.8 Dec 11, 2025
230 of 235 checks passed
@alsugiliazova
Copy link
Member

alsugiliazova commented Dec 17, 2025

Was not able to reproduce this issue, but found that partition_key and sorting_key are not exposed when using glue: #1235

I checked the code, relevant_snapshot variable is used safely now everywhere.

In case of rest, for the reproduction I tried to remove current_snapshot_id field from metadata.json or set it to 1.
In this case SHOW TABLES just returned 0 rows.

In case of glue, SHOW TABLES does not go to object storage, but uses glue metadata. I tried to modify some fields there (Parameters: metadata_location), or just corrupt it, but apparently SHOW TABLES only need table name and namespace to work correctly.

@alsugiliazova alsugiliazova added the verified Verified by QA label Dec 17, 2025
@svb-alt svb-alt added the port-antalya PRs to be ported to all new Antalya releases label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

antalya antalya-25.8 bugfix port-antalya PRs to be ported to all new Antalya releases verified Verified by QA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants